Skip to content

test: fix flaky request queue integration tests#665

Merged
vdusek merged 1 commit intomasterfrom
test/fix-request-queue-flaky-tests
Mar 10, 2026
Merged

test: fix flaky request queue integration tests#665
vdusek merged 1 commit intomasterfrom
test/fix-request-queue-flaky-tests

Conversation

@vdusek
Copy link
Contributor

@vdusek vdusek commented Mar 10, 2026 •
edited
Loading

Summary

  • Replace fixed 1-second sleeps with retry polling (up to 5 attempts) for eventual consistency in request queue integration tests
  • Failing CI run

Test plan

  • CI passes

@vdusek @claude
test: fix flaky request queue integration tests
05bc81f 
Replace fixed 1-second sleeps with retry polling (up to 5 attempts) for
eventual consistency in request queue tests. The single sleep was not
always sufficient, causing intermittent failures in CI.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vdusek vdusek added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels Mar 10, 2026
@vdusek vdusek self-assigned this Mar 10, 2026
@vdusek vdusek requested a review from Pijukatel March 10, 2026 09:59
@codecov
Copy link

codecov bot commented Mar 10, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.83%. Comparing base (723ec6e) to head (05bc81f).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files 
@@           Coverage Diff           @@
##           master     #665   +/-   ##
=======================================
  Coverage   94.83%   94.83%           
=======================================
  Files          45       45           
  Lines        4340     4340           
=======================================
  Hits         4116     4116           
  Misses        224      224
Flag Coverage Δ
integration 94.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek merged commit b0b2dac into master Mar 10, 2026
26 checks passed
@vdusek vdusek deleted the test/fix-request-queue-flaky-tests branch March 10, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Pijukatel Pijukatel Pijukatel approved these changes

Assignees

@vdusek vdusek

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vdusek @Pijukatel